Just develop to master #282
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initial plan * Fix Filament Blade component references in settings view - Replace filament-panels::form with filament::form - Replace filament-panels::form.actions with filament::form.actions - This fixes ViewException in SettingsTest tests Co-authored-by: nielsdrost7 <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: nielsdrost7 <[email protected]>
) * Initial plan * Fix logging, enum injection, and enum logic issues Co-authored-by: nielsdrost7 <[email protected]> * Update BaseFormatHandler to support both constructor and setFormat patterns Co-authored-by: nielsdrost7 <[email protected]> * Fix test base classes to use AbstractTestCase Co-authored-by: nielsdrost7 <[email protected]> * Address code review feedback - improve format property handling Co-authored-by: nielsdrost7 <[email protected]> * Address PR review feedback: import RuntimeException, fix test comments, remove empty test, clarify header merge order Co-authored-by: nielsdrost7 <[email protected]> * style: apply Laravel Pint fixes --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: nielsdrost7 <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…tion, and update AI guidelines (#354) * Initial plan * Fix HTTP client to not throw automatically, add config defaults, protect FormatHandlerFactory Co-authored-by: nielsdrost7 <[email protected]> * Fix PeppolService to validate invoices and check response status before returning Co-authored-by: nielsdrost7 <[email protected]> * Fix FormatHandlerFactory to only support subset of formats and gracefully fallback Co-authored-by: nielsdrost7 <[email protected]> * Fix Invoice and Quote observers to throw RuntimeException for duplicate numbers Co-authored-by: nielsdrost7 <[email protected]> * Restore ->throw() in ApiClient and let HttpClientExceptionHandler catch exceptions Co-authored-by: nielsdrost7 <[email protected]> * Add selective exception handling in DocumentsClient for test compatibility Co-authored-by: nielsdrost7 <[email protected]> * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> * Add documentation and logging for validation and format handler fallbacks Co-authored-by: nielsdrost7 <[email protected]> * Restore all Peppol format handlers with comprehensive PHPUnit tests Co-authored-by: nielsdrost7 <[email protected]> * Consolidate documentation: Enhanced README with comprehensive user guide Co-authored-by: nielsdrost7 <[email protected]> * Update guidelines and copilot instructions with complete Peppol format handler documentation Co-authored-by: nielsdrost7 <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: nielsdrost7 <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Contributor
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…n permissions (#360) Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…n permissions (#359) Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…n permissions (#358) Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…n permissions (#357) Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…n permissions (#356) Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.